Skip to content

Add EC lifecycle integration test scenarios#564

Closed
ChristianPavilonis wants to merge 1 commit intofeature/ec-pull-sync-dispatchfrom
feature/ec-integration-tests
Closed

Add EC lifecycle integration test scenarios#564
ChristianPavilonis wants to merge 1 commit intofeature/ec-pull-sync-dispatchfrom
feature/ec-integration-tests

Conversation

@ChristianPavilonis
Copy link
Collaborator

Summary

  • Add end-to-end EC lifecycle scenarios to the integration test matrix (full lifecycle, consent withdrawal, identify no-EC/denied, concurrent pixel syncs, batch sync happy path, and batch sync auth rejection).
  • Add EC-specific integration helpers in tests/common/ec.rs, including a cookie-aware test client and a minimal local origin server for organic-route proxying.
  • Add required Viceroy KV fixtures (consent_store, ec_identity_store, ec_partner_store) and update integration build env vars to use TRUSTED_SERVER__EC__PASSPHRASE.

Notes

  • EC scenarios run against a single Viceroy instance; KV state is intentionally shared across scenarios.
  • Pull-sync E2E, rate-limit E2E, and explicit KV-failure E2E remain out of scope for this PR (covered by unit tests / platform constraints).

Validation

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo test --workspace
  • cargo test --manifest-path crates/integration-tests/Cargo.toml --target $(rustc -vV | sed -n 's/^host: //p') -- --include-ignored test_ec_lifecycle_fastly

Closes #544

Implement Story 11 (#544): Viceroy-driven E2E tests covering full EC
lifecycle (generation, pixel sync, identify, batch sync, consent
withdrawal, auth rejection). Adds EC test helpers with manual cookie
tracking, minimal origin server with graceful shutdown, and required
KV store fixtures. Fixes integration build env vars.
@ChristianPavilonis ChristianPavilonis force-pushed the feature/ec-pull-sync-dispatch branch from a4ae046 to 3a9da2d Compare March 26, 2026 00:00
@ChristianPavilonis ChristianPavilonis force-pushed the feature/ec-integration-tests branch from 77d9f78 to 6c01e72 Compare March 26, 2026 00:00
@ChristianPavilonis
Copy link
Collaborator Author

Superseded by #582 which consolidates the full EC epic into a single PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant